Fix remote paths with tilde (~) for shell expansion#16
Merged
nitrobass24 merged 4 commits intomasterfrom Jan 27, 2026
Merged
Conversation
The remote scanner now properly handles paths containing `~` by converting them to `$HOME` for shell expansion. This allows users whose SSH and LFTP paths differ (e.g., LFTP locked to home directory) to use tilde paths. Changes: - Added `_escape_remote_path_single()` for literal paths (single quotes) - Added `_escape_remote_path_double()` for tilde paths (double quotes + $HOME) - Scan command uses consistent quoting based on whether path has tilde - Added unit test for tilde path handling Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The parser now gracefully handles empty or unexpected output from lftp `jobs -v` command instead of crashing with "Missing queue header line 1". Changes: - __parse_queue now returns empty list instead of raising on missing/invalid headers - Added tests for empty jobs command output scenarios Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Builds and pushes Docker image tagged 'develop' to ghcr.io on every push to the develop branch. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Fix parser failing with "First line is not a matching header" when LFTP output contains terminal control sequences like bracketed paste mode (^[[?2004l, ^[[?2004h). These ANSI escape codes are now stripped before parsing. Fixes #15 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
~by converting to$HOMEfor shell expansionjobs -voutputFixes #14
Fixes #15
Test plan
~/downloads/cross-seedpath